New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix build with clang v15.0.6 [-Wunused-but-set-variable] #1247
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this change fixes Squid build somewhere, please update PR title/description accordingly (e.g., see #1246).
I polished PR title, primarily to highlight that this is a code maintenance change (i.e. it does not affect Squid functionality). I removed PR description to fix its formatting -- the PR title now contains essentially the same info. Please review and adjust further as needed.
FWIW, I recommend avoiding key: text
PR title format except for a few commonly used keys like Maintenance
, Bug NNN
, Docs
, Language
, and CI
. Having a small set of commit title keys helps navigating regular git log
output. On the other hand, using function names as keys is just a distraction/noise -- folks searching for function X changes will use git log -L...
or similar tools tailored for that purpose.
clang v15.0.6 distributed with Fedora Rawhide complains about a variable that is updated but never used: net_db.cc:230:9: error: variable 'removed' set but not used [-Werror,-Wunused-but-set-variable]`
…#1247) clang v15.0.6 distributed with Fedora Rawhide complains about a variable that is updated but never used: net_db.cc:230:9: error: variable 'removed' set but not used [-Werror,-Wunused-but-set-variable]`
clang v15.0.6 distributed with Fedora Rawhide complains about a variable that is updated but never used: net_db.cc:230:9: error: variable 'removed' set but not used [-Werror,-Wunused-but-set-variable]`
clang v15.0.6 distributed with Fedora Rawhide complains about a variable
that is updated but never used: